-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update doc model_diva.py #332
Conversation
Codecov Report
@@ Coverage Diff @@
## master #332 +/- ##
=======================================
Coverage 95.20% 95.20%
=======================================
Files 129 129
Lines 4464 4464
=======================================
Hits 4250 4250
Misses 214 214
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very good!
domainlab/models/model_diva.py
Outdated
zd_dim: size of latent space for domain-specific information, | ||
zy_dim: size of latent space for class-specific information, | ||
zx_dim: size of latent space for residual variance, | ||
chain_node_builder: TODO, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can either first merge the new node handler you are implmenting into master, then update this branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I will first complete the test file and then update the doc again after merge.
No description provided.